Skip to content

Document deployment.stagingRetention.maxCount and payloadRetention.maxSize - #668

Open
dawsontoth wants to merge 1 commit into
mainfrom
docs/deploy-staging-retention
Open

Document deployment.stagingRetention.maxCount and payloadRetention.maxSize#668
dawsontoth wants to merge 1 commit into
mainfrom
docs/deploy-staging-retention

Conversation

@dawsontoth

Copy link
Copy Markdown
Contributor

Companion to HarperFast/harper#2531 (step 4 of HarperFast/harper#2315).

Adds a deployment section to the configuration reference covering the new stagingRetention.maxCount knob and the existing but previously undocumented payloadRetention.maxSize, and a 5.3 release note for staged build retention.

Verified with npm run format:check; no new pages or links beyond the existing deploy_component anchor.

…xSize

Adds a deployment section to the configuration reference for the two
retention knobs, and a 5.3 release note for staged build retention
(HarperFast/harper#2315 step 4).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request documents the new deployment configuration options (payloadRetention.maxSize and stagingRetention.maxCount) and updates the v5.3 release notes to detail the staged build retention feature. The feedback suggests formatting adjustments to adhere to style guidelines, including using hyphens instead of em dashes as field separators in the configuration options documentation, and removing direct links to internal GitHub issues in the user-facing release notes.

Comment thread reference/configuration/options.md
Comment thread release-notes/v5-lincoln/5.3.md
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

🚀 Preview Deployment

Your preview deployment is ready!

🔗 Preview URL: https://preview.harper-documentation.harperfabric.com/pr-668

This preview will update automatically when you push new commits.

@dawsontoth
dawsontoth marked this pull request as ready for review September 8, 2026 18:27
@dawsontoth
dawsontoth requested a review from a team as a code owner September 8, 2026 18:27

@kriszyp kriszyp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep!
🤖 Reviewed with Codex

maxCount: 5
```

- `payloadRetention.maxSize` — Bytes. After a successful deploy, a payload larger than this has its stored tarball (`payload_blob`) dropped from the `hdb_deployment` row; the row and its metadata stay. Set it very high to retain every payload; _Default_: `10485760` (10 MiB)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

payloadRetention.maxSize is a public configuration surface newly documented here, but unlike the adjacent retention setting it has no VersionBadge. The v5 reference combines minor releases, so users cannot tell whether their installed version accepts this key. Add a badge for its actual shipping minor (use v5.3.0 if it ships with this change).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants